-
Notifications
You must be signed in to change notification settings - Fork 1.4k
GHA: Enable testing #9450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
GHA: Enable testing #9450
Conversation
81fce8b to
bc9daf1
Compare
c049c75 to
7c97af9
Compare
Instead of only building the tests, execute them as well. Also run against swiftbuild backend. Fixes: swiftlang#9427 Issue: rdar://165491718
7c97af9 to
0472150
Compare
|
Since the PR modifies Utilities/build-using-self, which is used by the test hosted pipelines. let's trigger them. @swift-ci test self hosted |
| "swift-test", | ||
| *global_args, | ||
| "--vv", | ||
| # "--vv", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: why turn off very verbose here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find it significantly increases the log, making it difficult for view the console logs in Jenkins - not sure how GitHub Actions handles the larges logs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, it's my understanding it was added by @daveinglis to help troubleshoot CI build failure in a PR.
9a0fdf5 to
57222a9
Compare
57222a9 to
33b3f73
Compare
This reverts commit 33b3f73.
Instead of only building the tests, execute them as well
Fixes: #9427
Issue: rdar://165491718
Depends on: #9406